Skip to content

Use content:encoded for descriptions#34

Merged
RobbieTheWagner merged 1 commit intomainfrom
content-encoded
Jan 19, 2026
Merged

Use content:encoded for descriptions#34
RobbieTheWagner merged 1 commit intomainfrom
content-encoded

Conversation

@RobbieTheWagner
Copy link
Member

@RobbieTheWagner RobbieTheWagner commented Jan 19, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features
    • Added support for enriched RSS feed content to provide more detailed episode information when available.
  • Style
    • Improved spacing and formatting in show notes list items for better readability.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Jan 19, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
starpod Ready Ready Preview, Comment Jan 19, 2026 7:01pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

The pull request adds support for the content_encoded field in RSS feed parsing, preferring it over description for episode content when available, and applies a minor CSS class adjustment to remove margins from paragraphs nested within list items in show notes.

Changes

Cohort / File(s) Summary
RSS Feed Content Handling
src/lib/rss.ts		 Added optional content_encoded field to FeedSchema; introduced fallback logic to prefer content_encoded over description for episode content assignment. No public API changes.
Show Notes Presentation
src/pages/[episode].astro		 Extended prose container class with [&_li>p]:m-0 selector to eliminate margins on paragraphs directly inside list items.

Poem

🐰 A feed that's twice as rich and deep,
With encoded content we shall keep,
And list items formatted just right,
No extra margins stealing sight,
Our RSS flows with pure delight! 📡

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding support for content_encoded in RSS feed items as the primary source for episode descriptions.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@RobbieTheWagner RobbieTheWagner merged commit ddb7aef into main Jan 19, 2026
6 checks passed
@RobbieTheWagner RobbieTheWagner deleted the content-encoded branch January 19, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@RobbieTheWagner